docs: fix stale facts across .ai/ and README - #179
Merged
Merged
Conversation
…ocol The "take four control shots" section still told the next investigator to vary pointer position, even though the same section had already shown that Claude Code never enables mouse tracking and xterm.js de-duplicates identical motion - the pointer could never have been the variable. What actually closed the question was arming the trace on an idle session with nobody at the keyboard: a 340s trace found CPR/DECXCPR at 59% of pty.input traffic, not the already-excluded SGR mouse reports at 20%. Replace the protocol with what worked, and flag the 10-code-point cap on the cp field as a reading pitfall. Also correct trigger-watcher.js's LOC estimate (~800 -> ~1050, measured).
- ~/workspace/switchboard doesn't exist on this checkout; it's C:\Serveur\switchboard on Windows 11. Fixed the two path references. - main.js is measured at ~2600 LOC now, not ~1850. - Invariants #1 and #2 (and half of #6) describe the Linux AppImage deployment specifically (appimagelauncherd, npm run build:linux, ~/Applications/). Labelled them as such instead of deleting the field-measured incidents they carry, and noted the Windows equivalent is unverified rather than implicitly safe. - Fixed a broken relative link: [README.md](README.md) from inside .ai/ resolves to .ai/README.md, which doesn't exist; needed ../README.md like the other README link in this same file already has.
preload.js is 152 lines (was ~130), main.js is ~2600 (was ~1850) - both measured with wc -l. The session-forked send in session-transitions.js is at line 427, not 336; the code moved since this was written.
The file has moved on since these were written: - the routing branch (dataset.subagent && session.parentSessionId) is now at sidebar.js:1082, not :771 - grid-view.js/sidebar.js load at index.html:135/:139, not :132/:136 Verified against the current tree; the surrounding claims (script load order, dataset discriminator) still hold.
public/viewer-panel.js is 414 lines (was ~365, measured). The Ctrl+S Save-Page-prevention keydown listener in viewer-toolbar.js is at line 256, not 230 - checked with wc -l / grep against the current file.
…ne ref schedule-runner.js is 311 lines now (was ~220). The startScheduler() call site in main.js is at line 2517, not 1618 (main.js has grown substantially); kept the existing "or wherever" hedge since this number will keep moving.
All five were stale, measured with wc -l against the current tree: db.js ~450 -> ~895, session-cache.js ~525 -> ~690, read-session-file.js ~280 -> ~420, encode-project-path.js 14 -> 28 (was cited as an exact count, not an estimate), derive-project-path.js 64 -> ~155.
"~150 lines each" hasn't been true for a while - the seven docs range 76 to 526 lines (measured with wc -l). main.js is ~2600 LOC now, not 1800.
CLAUDE.md at the repo root is one line: @.ai/shared-guidelines.md. Both README.md and docs/README.md pointed AI agents at CLAUDE.md as though it documented fork conventions and invariants directly. Point at .ai/shared-guidelines.md, the file that actually does, and note the include relationship instead of leaving it implicit.
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Nine commits, one per file (or coherent group), each fixing documentation that no longer matches the codebase. No code changes -
.mdfiles only.Why
.ai/shared-guidelines.mdand.ai/contexts/*.mdare what AI agents working on this repo build on without re-deriving. Stale line numbers, LOC counts, and an outdated investigation protocol in there cost more than an omission would, because they're read as ground truth.Changes
pty.inputtraffic (PR fix(triggers): exclude CPR/DECXCPR from the composer quiet clock #170, open). Also fixed the file's own LOC estimate.~/workspace/switchboarddoesn't exist on this checkout (it'sC:\Serveur\switchboard, Windows 11);main.jsis ~2600 LOC now, not ~1850; two invariants written against the Linux AppImage deployment are now labelled as such instead of being read as universal; fixed a broken relative link (README.mdfrom inside.ai/needs../README.md).wc -l/grep -nagainst the checked-out code).CLAUDE.mdas if it held the content itself - it's a one-line@.ai/shared-guidelines.mdinclude; pointed at the file that actually documents the fork conventions.Not fixed here (flagged for a separate issue, not doc)
Taskfile.yaml:47still saystask testruns "(24 tests)" - the suite is at ~900 now. Not a.mdfile, out of scope for this PR..ai/project.json:36still says~/workspace/switchboard. Same reason.Verification
wc -lagainst this branch's checkout.grep -nagainst the actual current line.devsuitup/switchboard), not just asserted..ai/shared-guidelines.md's "Quick orientation" table links anddocs/README.md's page list were all checked to resolve to existing files - no changes needed there.